Skip to content

Fix caching issue in the metadata-service auth type.#1225

Merged
renaudhartert-db merged 6 commits into
mainfrom
renaud-hartert_data/cache
May 23, 2025
Merged

Fix caching issue in the metadata-service auth type.#1225
renaudhartert-db merged 6 commits into
mainfrom
renaud-hartert_data/cache

Conversation

@renaudhartert-db

Copy link
Copy Markdown
Contributor

What changes are proposed in this pull request?

This PR fixes a bug introduced in PR #1223 which led token to not be cached anymore when using the metadata-service auth type. The PR also slightly cleans up some authentication code by consolidating on using Databricks' version of the TokenSource.

How is this tested?

Unit and integration tests.

Comment thread NEXT_CHANGELOG.md
@github-actions

Copy link
Copy Markdown

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/sdk-go

Inputs:

  • PR number: 1225
  • Commit SHA: 4d94cb9f687c0acaec59dcf2a54f3d9b3ad3230b

Checks will be approved automatically on success.

@renaudhartert-db renaudhartert-db added this pull request to the merge queue May 23, 2025
Merged via the queue into main with commit c54b388 May 23, 2025
15 checks passed
@renaudhartert-db renaudhartert-db deleted the renaud-hartert_data/cache branch May 23, 2025 16:17
deco-sdk-tagging Bot added a commit that referenced this pull request May 26, 2025
## Release v0.71.0

### Bug Fixes

- Properly cache tokens when using the `metadata-service` auth type ([#1225](#1225)).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants